Skip to content

Add documentation correctly for nova sonic#20320

Merged
Sameerlite merged 1 commit intomainfrom
litellm_nova-sonic_doc
Feb 3, 2026
Merged

Add documentation correctly for nova sonic#20320
Sameerlite merged 1 commit intomainfrom
litellm_nova-sonic_doc

Conversation

@Sameerlite
Copy link
Collaborator

Relevant issues

Pre-Submission checklist

Please complete all items before asking a LiteLLM maintainer to review your PR

  • I have Added testing in the tests/litellm/ directory, Adding at least 1 test is a hard requirement - see details
  • My PR passes all unit tests on make test-unit
  • My PR's scope is as isolated as possible, it only solves 1 specific problem

CI (LiteLLM team)

CI status guideline:

  • 50-55 passing tests: main is stable with minor issues.
  • 45-49 passing tests: acceptable but needs attention
  • <= 40 passing tests: unstable; be careful with your merges and assess the risk.
  • Branch creation CI run
    Link:

  • CI run for the last commit
    Link:

  • Merge / cherry-pick CI run
    Links:

Type

🆕 New Feature
🐛 Bug Fix
🧹 Refactoring
📖 Documentation
🚄 Infrastructure
✅ Test

Changes

@vercel
Copy link

vercel bot commented Feb 3, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
litellm Ready Ready Preview, Comment Feb 3, 2026 3:35am

Request Review

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Feb 3, 2026

Greptile Overview

Greptile Summary

Moved Bedrock Nova Sonic realtime API documentation from tutorials to providers section and updated the main bedrock provider page to list /v1/realtime as a supported endpoint.

Changes:

  • Relocated bedrock_realtime_with_audio.md from docs/tutorials/ to docs/providers/ for better organization
  • Simplified document title from "Call Bedrock Nova Sonic Realtime API with Audio Input/Output" to "Bedrock Realtime API"
  • Removed version requirement info box (Requires LiteLLM Proxy v1.70.1+)
  • Added /v1/realtime to the supported OpenAI endpoints table in bedrock.md
  • Updated sidebar navigation to include the relocated documentation

Notes:

  • Minor formatting inconsistency in bedrock.md (missing space before pipe separator)

Confidence Score: 5/5

  • Safe to merge - documentation-only changes with proper organization improvements
  • This is a straightforward documentation reorganization PR. The changes properly move realtime API docs to the correct location, update navigation, and add the endpoint to the supported endpoints list. Only one minor style issue found (formatting)
  • No files require special attention - all changes are documentation updates

Important Files Changed

Filename Overview
docs/my-website/docs/providers/bedrock.md Added /v1/realtime endpoint to supported OpenAI endpoints list with minor formatting issue (missing space before pipe)
docs/my-website/docs/providers/bedrock_realtime_with_audio.md Moved from tutorials to providers directory, simplified header from detailed title to generic "Bedrock Realtime API", removed version requirement info box
docs/my-website/sidebars.js Added bedrock_realtime_with_audio entry to bedrock providers section in sidebar navigation

Sequence Diagram

sequenceDiagram
    participant Dev as Developer
    participant Docs as Documentation Site
    participant Sidebar as Sidebar Navigation
    participant Bedrock as Bedrock Provider Page
    
    Note over Dev,Bedrock: Documentation Reorganization
    
    Dev->>Docs: Move bedrock_realtime_with_audio.md
    Note right of Docs: From: docs/tutorials/<br/>To: docs/providers/
    
    Dev->>Docs: Simplify document header
    Note right of Docs: Remove version requirement<br/>Simplify title
    
    Dev->>Sidebar: Add navigation entry
    Sidebar->>Sidebar: bedrock_realtime_with_audio added to providers
    
    Dev->>Bedrock: Update supported endpoints
    Bedrock->>Bedrock: Add /v1/realtime to table
    
    Note over Dev,Bedrock: Result: Realtime API docs properly organized in providers section
Loading

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

3 files reviewed, 1 comment

Edit Code Review Agent Settings | Greptile

| Provider Route on LiteLLM | `bedrock/`, [`bedrock/converse/`](#set-converse--invoke-route), [`bedrock/invoke/`](#set-invoke-route), [`bedrock/converse_like/`](#calling-via-internal-proxy), [`bedrock/llama/`](#deepseek-not-r1), [`bedrock/deepseek_r1/`](#deepseek-r1), [`bedrock/qwen3/`](#qwen3-imported-models), [`bedrock/qwen2/`](./bedrock_imported.md#qwen2-imported-models), [`bedrock/openai/`](./bedrock_imported.md#openai-compatible-imported-models-qwen-25-vl-etc), [`bedrock/moonshot`](./bedrock_imported.md#moonshot-kimi-k2-thinking) |
| Provider Doc | [Amazon Bedrock ↗](https://docs.aws.amazon.com/bedrock/latest/userguide/what-is-bedrock.html) |
| Supported OpenAI Endpoints | `/chat/completions`, `/completions`, `/embeddings`, `/images/generations` |
| Supported OpenAI Endpoints | `/chat/completions`, `/completions`, `/embeddings`, `/images/generations`, `/v1/realtime`|
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing space before the pipe separator

Suggested change
| Supported OpenAI Endpoints | `/chat/completions`, `/completions`, `/embeddings`, `/images/generations`, `/v1/realtime`|
| Supported OpenAI Endpoints | `/chat/completions`, `/completions`, `/embeddings`, `/images/generations`, `/v1/realtime` |

Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!

Prompt To Fix With AI
This is a comment left during a code review.
Path: docs/my-website/docs/providers/bedrock.md
Line: 12:12

Comment:
Missing space before the pipe separator

```suggestion
| Supported OpenAI Endpoints | `/chat/completions`, `/completions`, `/embeddings`, `/images/generations`, `/v1/realtime` |
```

<sub>Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!</sub>

How can I resolve this? If you propose a fix, please make it concise.

@Sameerlite Sameerlite merged commit 24a4979 into main Feb 3, 2026
37 of 56 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant